Skip to content

Set libtool tag per command instead of global one#12350

Closed
jpalus wants to merge 1 commit into
php:PHP-8.1from
jpalus:libtool-tag
Closed

Set libtool tag per command instead of global one#12350
jpalus wants to merge 1 commit into
php:PHP-8.1from
jpalus:libtool-tag

Conversation

@jpalus

@jpalus jpalus commented Oct 2, 2023

Copy link
Copy Markdown
Contributor

Global --tag=CC defined in configure.ac is not correct in all cases. For example linking objects that were compiled from C++ sources needs to be done with C++ compiler, however for link mode libtool will prefer compiler indicated with --tag.

Fixes GH-12349

Global --tag=CC defined in configure.ac is not correct in all cases. For example
linking objects that were compiled from C++ sources needs to be done with C++
compiler, however for link mode libtool will prefer compiler indicated with
--tag.

Fixes phpGH-12349
@iluuu1994
iluuu1994 requested a review from petk October 3, 2023 09:20
pld-gitsync pushed a commit to pld-linux/php that referenced this pull request Oct 26, 2023
@petk

petk commented Jan 6, 2024

Copy link
Copy Markdown
Member

I'm onto this in the following days, thanks for your patience... Mold looks very promising and PHP definitely should support it also.

@petk petk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for PHP-8.2 branch. Those PHP_COMPILE and CXX_PHP_COMPILE variables aren't used in the code anymore but it's fine if they are changed also. Thanks @jpalus

@petk

petk commented Jan 10, 2024

Copy link
Copy Markdown
Member

Applied to PHP-8.2 branch and up via d57a776

@petk petk closed this Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants